refactor: refactor sharded wal to handle coordinator#1237
refactor: refactor sharded wal to handle coordinator#1237
Conversation
|
View your CI Pipeline Execution ↗ for commit 674fe70
☁️ Nx Cloud last updated this comment at |
…rdinator-in-sharded-wal
# Conflicts: # packages/utils/src/lib/wal.ts # packages/utils/src/lib/wal.unit.test.ts
…-sharded-wal' into feat/utils/handle-coordinator-in-sharded-wal
@code-pushup/ci
@code-pushup/cli
@code-pushup/core
@code-pushup/create-cli
@code-pushup/models
@code-pushup/nx-plugin
@code-pushup/axe-plugin
@code-pushup/coverage-plugin
@code-pushup/eslint-plugin
@code-pushup/js-packages-plugin
@code-pushup/jsdocs-plugin
@code-pushup/lighthouse-plugin
@code-pushup/typescript-plugin
@code-pushup/utils
commit: |
Code PushUp🤨 Code PushUp report has both improvements and regressions – compared current commit 069d350 with previous commit 4c80af5. 🕵️ See full comparison in Code PushUp portal 🔍 🏷️ Categories👍 2 groups improved, 👎 1 group regressed, 👍 9 audits improved, 👎 3 audits regressed, 15 audits changed without impacting score🗃️ Groups
31 other groups are unchanged. 🛡️ Audits
652 other audits are unchanged. |
Code PushUp🤨 Code PushUp report has both improvements and regressions – compared current commit 069d350 with previous commit 4c80af5. 💼 Project
|
| 🏷️ Category | ⭐ Previous score | ⭐ Current score | 🔄 Score change |
|---|---|---|---|
| Documentation | 🟡 60 | 🟡 61 | |
| Code coverage | 🟢 95 | 🟢 95 |
4 other categories are unchanged.
👍 1 group improved, 👎 1 group regressed, 👍 5 audits improved, 👎 3 audits regressed
🗃️ Groups
| 🔌 Plugin | 🗃️ Group | ⭐ Previous score | ⭐ Current score | 🔄 Score change |
|---|---|---|---|---|
| JSDocs coverage | Documentation coverage | 🟡 60 | 🟡 61 | |
| Code coverage | Code coverage metrics | 🟢 95 | 🟢 95 |
13 other groups are unchanged.
🛡️ Audits
| 🔌 Plugin | 🛡️ Audit | 📏 Previous value | 📏 Current value | 🔄 Value change |
|---|---|---|---|---|
| JSDocs coverage | Methods coverage | 🟨 10 undocumented methods | 🟨 10 undocumented methods | |
| JSDocs coverage | Properties coverage | 🟥 40 undocumented properties | 🟥 44 undocumented properties | |
| Code coverage | Branch coverage | 🟩 91.8 % | 🟩 91.2 % | |
| JSDocs coverage | Types coverage | 🟨 55 undocumented types | 🟨 56 undocumented types | |
| JSDocs coverage | Variables coverage | 🟥 49 undocumented variables | 🟥 50 undocumented variables | |
| Code coverage | Function coverage | 🟩 95.8 % | 🟩 95.9 % | |
| JSDocs coverage | Functions coverage | 🟥 243 undocumented functions | 🟥 242 undocumented functions | |
| Code coverage | Line coverage | 🟩 97.8 % | 🟩 97.8 % |
436 other audits are unchanged.
💼 Project plugin-lighthouse
🤨 Code PushUp report has both improvements and regressions.
🕵️ See full comparison in Code PushUp portal 🔍
All of 6 categories are unchanged.
1 audit changed without impacting score
🗃️ Groups
All of 15 groups are unchanged.
🛡️ Audits
| 🔌 Plugin | 🛡️ Audit | 📏 Previous value | 📏 Current value | 🔄 Value change |
|---|---|---|---|---|
| Code coverage | Branch coverage | 🟩 98.8 % | 🟩 98.8 % |
443 other audits are unchanged.
12 other projects are unchanged.
Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>
Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>
Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>
Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>
Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>
…rdinator-in-sharded-wal
| }); | ||
|
|
||
| it('repacks with invalid entries and logs warning', () => { | ||
| it('repacks with without invalid entries and logs warning', () => { |
There was a problem hiding this comment.
| it('repacks with without invalid entries and logs warning', () => { | |
| it('repacks without invalid entries and logs warning', () => { |
| * @param opt.groupId - Group ID for sharding (defaults to generated group ID) | ||
| * @param opt.coordinatorIdEnvVar - Environment variable name for storing coordinator ID (defaults to CP_SHARDED_WAL_COORDINATOR_ID) | ||
| * @param opt.autoCoordinator - Whether to auto-set the coordinator ID on construction (defaults to true) | ||
| * @param opt.measureNameEnvVar - Environment variable name for coordinating groupId across processes (optional) |
There was a problem hiding this comment.
The constructor signature doesn't have a measureNameEnvVar parameter. It is, however, included in tests.
| export function extendError( | ||
| error: unknown, | ||
| message: string, | ||
| { appendMessage = false } = {}, |
There was a problem hiding this comment.
The message is prepended, while the cause is appended 🤔
Precondition:
Related:
This PR includes:
Followup: